Skip to content

feat: AI integration baseline#420

Open
gabitoesmiapodo wants to merge 4 commits intomainfrom
feat/ai-integration
Open

feat: AI integration baseline#420
gabitoesmiapodo wants to merge 4 commits intomainfrom
feat/ai-integration

Conversation

@gabitoesmiapodo
Copy link
Collaborator

@gabitoesmiapodo gabitoesmiapodo commented Mar 23, 2026 •
edited
Loading

Summary

This branch adds AI agent context files (CLAUDE.md / AGENTS.md) and serves as the destination branch for a complete web quality initiative: a test safety net (5 PRs) followed by a web audit fix chain (5 PRs), all stacked incrementally so tests catch any regressions introduced by fixes.

This branch

  • CLAUDE.md + AGENTS.md — project instructions for AI agents (Claude and others). Both files are kept in sync; AGENTS.md is a regular file copy of CLAUDE.md.
  • .claude/ — Claude Code settings and ignore rules

Stacked PR chain (merge in order)

Test safety net

# Branch Target Scope
#415 test/utils feat/ai-integration Tier 1: Pure utility + env tests
#416 test/enhance-existing test/utils Tier 2: Enhanced BigNumberInput + HashInput tests
#417 test/hooks test/enhance-existing Tier 3: useWeb3Status, useTokenLists, useErc20Balance hook tests
#418 test/components test/hooks Tier 4: TransactionButton, SwitchNetwork, WalletStatusVerifier, suspenseWrapper tests
#419 test/demo-smoke test/components Tier 5: Demo page smoke tests

Web quality fixes (stacked on top of tests)

# Branch Target Scope
#421 fix/a11y test/demo-smoke WCAG AA color contrast, skip link, focus rings, aria attributes
#422 fix/performance fix/a11y Lazy loading, dns-prefetch, vendor chunk splitting
#423 fix/seo fix/performance JSON-LD structured data, sitemap hostname env var
#424 fix/security fix/seo CSP report-only mode + security headers via vercel.json
#425 fix/deps fix/security Minor/patch dependency updates including security vulnerability fixes

What was fixed

Accessibility

  • Warning color #cc0 → #996600 (light) / #e6b800 (dark) — was 1.3:1 contrast, now ~6-7:1
  • Danger and OK tokens now have separate light/dark values to pass WCAG AA in both modes
  • Skip link added to root layout for keyboard navigation
  • aria-invalid added to BigNumberInput on validation error
  • _focusVisible focus rings restored on CopyButton and ExternalLink (previously outline: none)
  • aria-label added to all icon-only buttons (close buttons, explorer links)

Performance

  • loading="lazy" on TokenLogo and Avatar images
  • DNS prefetch hints for RPC endpoints in index.html
  • Vite manualChunks for React, wagmi/viem, TanStack, Chakra UI, and AppKit vendor bundles

SEO

  • JSON-LD structured data (Organization + WebApplication schemas) in index.html
  • Sitemap hostname now reads from PUBLIC_APP_URL env var with fallback

Security

  • Content-Security-Policy-Report-Only header in vercel.json (non-breaking, report-only mode)
  • X-Content-Type-Options: nosniff, X-Frame-Options: DENY, Referrer-Policy: strict-origin-when-cross-origin

Dependencies

  • Updated all minor/patch packages (react, viem, chakra-ui, tanstack, reown/appkit, etc.)
  • Fixes h3 path traversal vulnerability via @reown/appkit 1.8.19

Merge order

#415 → #416 → #417 → #418 → #419 → #421 → #422 → #423 → #424 → #425 → #420 (into main)

Test plan

  • All 10 stacked PRs merged in order
  • pnpm test:coverage confirms no regressions
  • pnpm build passes cleanly
  • Lighthouse audit run on preview — target 90+ in all categories
  • Keyboard navigation tested: all interactive elements reachable with visible focus

Copilot AI review requested due to automatic review settings March 23, 2026 21:18
@vercel
Copy link

vercel bot commented Mar 23, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
components.dappbooster Ready Ready Preview, Comment Mar 23, 2026 10:22pm
demo.dappbooster Ready Ready Preview Mar 23, 2026 10:22pm
docs.dappbooster Ready Ready Preview, Comment Mar 23, 2026 10:22pm

Request Review

Copilot AI reviewed Mar 23, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds baseline documentation and configuration files intended to support AI-assisted workflows (Claude Code) alongside the existing dAppBooster template, plus local-ignore rules for Claude-specific files.

Changes:

  • Add CLAUDE.md with repo setup, architecture, conventions, and testing guidance.
  • Add Claude Code configuration (.claude/settings.json) and ignore local Claude artifacts via .gitignore.
  • Add AGENTS.md pointing at CLAUDE.md (likely as a symlink-style pointer).

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
CLAUDE.md Adds repo “operator’s manual” covering setup, structure, conventions, and patterns.
AGENTS.md Adds an agent entry pointing to CLAUDE.md (potentially via symlink representation).
.gitignore Ignores Claude Code local settings and local CLAUDE notes.
.claude/settings.json Allows web fetch access to specific documentation domains.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gabitoesmiapodo gabitoesmiapodo self-assigned this Mar 23, 2026
@gabitoesmiapodo
docs: fix pipe separator in CLAUDE.md and convert AGENTS.md to regula…
5f33ece 
…r file

Replace the literal pipe character between the two URLs in the header
with a space separator to avoid Markdown table syntax confusion.

Convert AGENTS.md from a symlink to a regular file copy of CLAUDE.md
so it works reliably on Windows and in CI environments where symlinks
may not be supported.
@gabitoesmiapodo gabitoesmiapodo requested a review from Copilot March 23, 2026 22:18
Copilot AI reviewed Mar 23, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

fernandomg
fernandomg approved these changes Mar 26, 2026
View reviewed changes
AGENTS.md
@@ -0,0 +1,231 @@
# dAppBooster

> This file is mirrored as `CLAUDE.md` for Claude. Keep both files in sync when making changes.
Copy link
Member

@fernandomg fernandomg Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are we going with this approach instead of the symlink?

Copy link
Collaborator Author

@gabitoesmiapodo gabitoesmiapodo Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd go with the symlink approach, but in this case this is a tool and it should work on as many platforms as possible. Symlinks are OK for Unix-like systems (which I assume is what most people use to develop web apps), but leave out people working on Windows so I guess this is a better way until a standard is adopted by everyone / most LLMs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@fernandomg fernandomg fernandomg approved these changes

Assignees

@gabitoesmiapodo gabitoesmiapodo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gabitoesmiapodo @fernandomg